-
Notifications
You must be signed in to change notification settings - Fork 707
Don't require VSCode ext host restart on change #1722
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Fixes microsoft/vscode#266087 The VSCode TS extension should now dynamically register/unregister language features so a restart isn't required for current insiders builds
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR removes the requirement for VSCode extension host restart when the TypeScript native preview configuration changes, improving the user experience for VSCode insiders builds 1.105 and later.
- Adds version detection logic to determine if extension host restart is needed
- Conditionally handles configuration changes based on VSCode version
- Maintains backward compatibility with older VSCode versions
I tried testing out the latest insiders; I have tsgo enabled on load, then disable it, tsserver does start but tsgo stays running (duplicating everything). If I enable tsgo again, tsserver's errors do not disappear. So, there might be more changes needed on both sides to better disable things. In our extension, we probably need to hook up disposasls and so on, and on the VS Code side, push diagnostics need to be cleared out? |
Our disposables are hooked up for deactivation, but a settings change to of |
The latest commit seems to do the trick, though is not perfect since |
Thanks @andrewbranch! Changes look good to me. Let's get this merged so we can do more self hosting |
Fixes microsoft/vscode#266087
The VSCode TS extension should now dynamically register/unregister language features so a restart isn't required for current insiders builds